x86: Fix XENPF_getidletime to correctly modify cpumask.
authorKeir Fraser <keir.fraser@citrix.com>
Wed, 3 Jun 2009 10:17:00 +0000 (11:17 +0100)
committerKeir Fraser <keir.fraser@citrix.com>
Wed, 3 Jun 2009 10:17:00 +0000 (11:17 +0100)
Signed-off-by: Keir Fraser <keir.fraser@citrix.com>
xen/arch/x86/platform_hypercall.c

index 4bf677792325d6af92f055d4b18c96b780554dbf..bb2f31380d32dd930abf098b84c394a58420dcc5 100644 (file)
@@ -313,7 +313,6 @@ ret_t do_platform_op(XEN_GUEST_HANDLE(xen_platform_op_t) u_xenpf_op)
     {
         uint32_t cpu;
         uint64_t idletime, now = NOW();
-        struct vcpu *v;
         struct xenctl_cpumap ctlmap;
         cpumask_t cpumap;
         XEN_GUEST_HANDLE(uint8) cpumap_bitmap;
@@ -336,7 +335,7 @@ ret_t do_platform_op(XEN_GUEST_HANDLE(xen_platform_op_t) u_xenpf_op)
 
         for_each_cpu_mask ( cpu, cpumap )
         {
-            if ( (v = idle_vcpu[cpu]) != NULL )
+            if ( idle_vcpu[cpu] == NULL )
                 cpu_clear(cpu, cpumap);
             idletime = get_cpu_idle_time(cpu);